-
Notifications
You must be signed in to change notification settings - Fork 1
merge walker #13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
merge walker #13
Conversation
Set up Python project "test-aoc"
Implement a CLI to interact with Advent of Code puzzles. New Features: - Added a command-line interface to download, read, and submit Advent of Code puzzle solutions, and update a results table in the README. Tests: - Added comprehensive tests for encoding/decoding, parsing, file operations, and CLI commands. Resolves #2
feat: Implement Advent of Code CLI
Add a GitHub Actions workflow to automatically merge pull requests from the Sourcery AI bot. New Features: - Introduce automated merging of Sourcery AI bot PRs using GitHub Actions. CI: - Set up a workflow to automatically merge pull requests opened by the `sourcery-ai[bot]` user. The workflow uses the `gh` CLI to merge the PR with the squash strategy, auto-merge, and delete the branch after merging. Resolves sourcery-ai#6 Co-authored-by: sourcery-ai[bot] <sourcery-ai[bot]@users.noreply.github.com>
CI: - Add a workflow to create issues for Advent of Code puzzles, including instructions for solution and test file placement, and a comment requesting automated analysis from Sourcery. Resolves sourcery-ai#3 Co-authored-by: sourcery-ai[bot] <sourcery-ai[bot]@users.noreply.github.com>
Add a GitHub Actions workflow to automatically submit Advent of Code solutions, update the README with results, and download the next part of the puzzle. New Features: - Automatically download the next part of the puzzle after successfully submitting the current part. Enhancements: - Update the README file with the latest Advent of Code results. CI: - Set up a new GitHub Actions workflow to submit Advent of Code puzzle solutions. Resolves sourcery-ai#5 Co-authored-by: sourcery-ai[bot] <sourcery-ai[bot]@users.noreply.github.com>
--- For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/JohnDaWalka/test-aoc?shareId=XXXX-XXXX-XXXX-XXXX).
Integrate code into GitHub project
Reviewer's Guide by SourceryThis pull request updates the repository link in the README file to point to the correct repository, ensuring that users are directed to the appropriate location for information and contributions. No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @JohnDaWalka - I've reviewed your changes - here's some feedback:
Overall Comments:
- Please ensure the pull request title is descriptive.
- The description should explain the purpose of the pull request.
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Review instructions: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mergin repos
* **create-puzzle-issue.yml** - Add a step to use `actions/checkout@v4` before creating issues - Update `GH_TOKEN` to use `${{ secrets.GH_TOKEN }}` * **merge-sourcery-ai-bot-pr.yml** - Add a step to use `actions/checkout@v4` before merging pull requests - Update `GH_TOKEN` to use `${{ secrets.GH_TOKEN }}` * **submit-advent-of-code.yml** - Update `GH_TOKEN` to use `${{ secrets.GH_TOKEN }}`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code integration
…kflows * **create-puzzle-issue.yml** - Add a step to merge `develop` into `main` after creating the puzzle issue - Use `gh pr merge` command with `--squash`, `--auto`, and `--delete-branch` options * **merge-sourcery-ai-bot-pr.yml** - Add a step to merge `develop` into `main` after merging the pull request - Use `gh pr merge` command with `--squash`, `--auto`, and `--delete-branch` options * **submit-advent-of-code.yml** - Add a step to merge `develop` into `main` after submitting the solution - Use `gh pr merge` command with `--squash`, `--auto`, and `--delete-branch` options * **README.md** - Update documentation to include information about new GitHub Actions workflows for merging code - Provide examples of how to use `gh pr merge` command with `--squash`, `--auto`, and `--delete-branch` options
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
using sorcery for poker coach
* **poker_coach/ui.py** - Create a new file to implement the interactive UI for the poker coach using Tkinter - Implement the necessary code to create the interactive UI with hand entry and analysis functionality * **README.md** - Add a section on how to use the poker coach UI - Include installation and usage instructions for the poker coach UI * **tests/test_poker_coach_ui.py** - Create a new file to add tests for the poker coach UI - Write tests to ensure the UI functions correctly
Summary by Sourcery
Documentation: